Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

release: Release 3 gems #513

Closed
wants to merge 1 commit into from
Closed

release: Release 3 gems #513

wants to merge 1 commit into from

Conversation

github-actions[bot]
Copy link
Contributor

This pull request prepares new gem releases for the following gems:

  • opentelemetry-instrumentation-redis 0.11.0 (was 0.10.1)
  • opentelemetry-instrumentation-mysql2 0.11.0 (was 0.10.1)
  • opentelemetry-instrumentation-dalli 0.11.0 (was 0.10.0)

For each gem, this pull request modifies the gem version and provides an initial changelog entry based on conventional commit messages. You can edit these changes before merging, to release a different version or to alter the changelog text.

  • To confirm this release, merge this pull request, ensuring the "release: pending" label is set. The release script will trigger automatically on merge.
  • To abort this release, close this pull request without merging.

The generated changelog entries have been copied below:

opentelemetry-instrumentation-redis

v0.11.0 / 2020-12-11

  • ADDED: Accept config for redis peer service attribute
  • ADDED: Move utf8 encoding to common utils

opentelemetry-instrumentation-mysql2

v0.11.0 / 2020-12-11

  • ADDED: Add peer service config to mysql

opentelemetry-instrumentation-dalli

v0.11.0 / 2020-12-11

  • ADDED: Add peer service config to dalli
  • ADDED: Move utf8 encoding to common utils

@github-actions github-actions bot added the release: pending Automated release is pending label Dec 11, 2020
@github-actions
Copy link
Contributor Author

WARNING: An additional commit was added while this release PR was open.
You may need to add to the changelog, or close this PR and prepare a new one.

Commit link: 754f90a

Message:
fix: copyright comments to not reference year (#512)

Co-authored-by: Francis Bogsanyi [email protected]

@robertlaurin
Copy link
Contributor

@fbogsany I think this release will break Dalli & Redis instrumentation.

Move utf8 encoding to common utils

Requires that their dependency on common get bumped along with getting common bumped so they have access to the utf8 encode.

@github-actions
Copy link
Contributor Author

WARNING: An additional commit was added while this release PR was open.
You may need to add to the changelog, or close this PR and prepare a new one.

Commit link: 87b8a47

Message:
feat: metrics reporting from trace export (#510)

  • feat: metrics reporting from trace export

  • fix: pluralize dropped_spans

  • fix: otel.bsp.export.spans -> ...exported_spans

@fbogsany
Copy link
Contributor

Requires that their dependency on common get bumped along with getting common bumped so they have access to the utf8 encode.

Could you please PR a fix for that? I can close this for now and open a new release PR when you're ready.

@fbogsany fbogsany closed this Dec 11, 2020
@github-actions github-actions bot added release: aborted Automated release was aborted and removed release: pending Automated release is pending labels Dec 11, 2020
@github-actions github-actions bot deleted the release/multi/20201211025801 branch December 11, 2020 15:50
@github-actions
Copy link
Contributor Author

Release PR closed without merging.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dazuma dazuma Awaiting requested review from dazuma

@ericmustin ericmustin Awaiting requested review from ericmustin

@fbogsany fbogsany Awaiting requested review from fbogsany

@mwear mwear Awaiting requested review from mwear

@robertlaurin robertlaurin Awaiting requested review from robertlaurin

Assignees
No one assigned
Labels
release: aborted Automated release was aborted
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@robertlaurin @fbogsany @dazuma